Skip to content

Documentation for checkVolume API#380

Merged
yadvr merged 4 commits intoapache:4.19from
shapeblue:CheckVolumeAPI
Feb 29, 2024
Merged

Documentation for checkVolume API#380
yadvr merged 4 commits intoapache:4.19from
shapeblue:CheckVolumeAPI

Conversation

@harikrishna-patnala
Copy link
Member

@harikrishna-patnala harikrishna-patnala commented Feb 5, 2024 •
edited by github-actions bot
Loading

This is a document PR for the PR apache/cloudstack#8577

📚 Documentation preview 📚: https://cloudstack-documentation--380.org.readthedocs.build/en/380/

@harikrishna-patnala harikrishna-patnala mentioned this pull request Feb 5, 2024
Merged
8 tasks
@harikrishna-patnala
Copy link
Member Author

harikrishna-patnala commented Feb 5, 2024

@blueorangutan docbuild

@blueorangutan
Copy link

blueorangutan commented Feb 5, 2024

@harikrishna-patnala a Jenkins job has been kicked to build the document. I'll keep you posted as I make progress.

@blueorangutan
Copy link

blueorangutan commented Feb 5, 2024

QA-Doc build preview: https://qa.cloudstack.cloud/builds/docs-build/pr/380. (QA-JID 205)

@harikrishna-patnala
Copy link
Member Author

harikrishna-patnala commented Feb 5, 2024

Here it is https://qa.cloudstack.cloud/builds/docs-build/pr/380/adminguide/storage.html#check-and-repair-volume

image

@harikrishna-patnala
Copy link
Member Author

harikrishna-patnala commented Feb 7, 2024

@blueorangutan docbuild

@blueorangutan
Copy link

blueorangutan commented Feb 7, 2024

@harikrishna-patnala a Jenkins job has been kicked to build the document. I'll keep you posted as I make progress.

@blueorangutan
Copy link

blueorangutan commented Feb 7, 2024

QA-Doc build preview: https://qa.cloudstack.cloud/builds/docs-build/pr/380. (QA-JID 211)

kiranchavala
kiranchavala reviewed Feb 20, 2024
View reviewed changes
Copy link
Member

@kiranchavala kiranchavala left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@harikrishna-patnala

Could you please mention the check volume api will work only if disk formats is in qcow2.

It doesn't work with Raw disk format

Also for Powerflex, the volumes need to be in encrypted format for it to work

@harikrishna-patnala
Copy link
Member Author

harikrishna-patnala commented Feb 20, 2024

@harikrishna-patnala

Could you please mention the check volume api will work only if disk formats is in qcow2.

It doesn't work with Raw disk format

Also for Powerflex, the volumes need to be in encrypted format for it to work

Added the volume format detail @kiranchavala that implicitly means about powerflex volumes as well. Please review.

sureshanaparti
sureshanaparti reviewed Feb 20, 2024
View reviewed changes
sureshanaparti
sureshanaparti reviewed Feb 20, 2024
View reviewed changes
sureshanaparti
sureshanaparti reviewed Feb 20, 2024
View reviewed changes
sureshanaparti
sureshanaparti reviewed Feb 20, 2024
View reviewed changes
@harikrishna-patnala
Copy link
Member Author

harikrishna-patnala commented Feb 20, 2024

Addressed them @sureshanaparti, please review

@harikrishna-patnala
Copy link
Member Author

harikrishna-patnala commented Feb 20, 2024

@blueorangutan docbuild

@blueorangutan
Copy link

blueorangutan commented Feb 20, 2024

@harikrishna-patnala a Jenkins job has been kicked to build the document. I'll keep you posted as I make progress.

sureshanaparti
sureshanaparti approved these changes Feb 20, 2024
View reviewed changes
Copy link
Contributor

@sureshanaparti sureshanaparti left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

doc updates lgtm

@blueorangutan
Copy link

blueorangutan commented Feb 20, 2024

QA-Doc build preview: https://qa.cloudstack.cloud/builds/docs-build/pr/380. (QA-JID 214)

kiranchavala
kiranchavala approved these changes Feb 20, 2024
View reviewed changes
Copy link
Member

@kiranchavala kiranchavala left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, doc contains the necessary information

@harikrishna-patnala harikrishna-patnala changed the base branch from main to 4.19 February 21, 2024 04:53
@harikrishna-patnala harikrishna-patnala added this to the 4.19.1 milestone Feb 21, 2024
@yadvr yadvr merged commit 5a47bef into apache:4.19 Feb 29, 2024
@yadvr yadvr deleted the CheckVolumeAPI branch February 29, 2024 09:10
DaanHoogland added a commit that referenced this pull request Apr 8, 2024
@DaanHoogland
Merge release branch 4.19 to main
f5fc6e7 
* 4.19:
  notes on POD-subnets (#391)
  Documentation for checkVolume API (#380)
  Document advance DRS settings (#374)
  veeam: update API commands and versions in doc (#357)
  Update virtual_machines.rst (#353)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@kiranchavala kiranchavala kiranchavala approved these changes

@sureshanaparti sureshanaparti sureshanaparti approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

4.19.1

Development

Successfully merging this pull request may close these issues.

5 participants

@harikrishna-patnala @blueorangutan @kiranchavala @sureshanaparti @yadvr